Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update changelog-3.4 to protection for rangePermCache with a RW lock #14231

Merged
merged 1 commit into from
Jul 24, 2022

Conversation

mitake
Copy link
Contributor

@mitake mitake commented Jul 18, 2022

Signed-off-by: Hitoshi Mitake h.mitake@gmail.com

Copy link
Member

@spzala spzala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but needs to wait for #14230

@@ -14,7 +14,8 @@ See [code changes](https://github.com/etcd-io/etcd/compare/v3.4.18...v3.4.19) an
- Fix [lease leak issue due to tokenProvider isn't enabled when restoring auth store from a snapshot](https://github.com/etcd-io/etcd/pull/13206).
- Fix [the race condition between goroutine and channel on the same leases to be revoked](https://github.com/etcd-io/etcd/pull/14150).
- Fix [lessor may continue to schedule checkpoint after stepping down leader role](https://github.com/etcd-io/etcd/pull/14150).

- Fix [Protect rangePermCache with a RW lock correctly](https://github.com/etcd-io/etcd/pull/14230)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mitake .

3.4.19 has already been released, this PR will be included in 3.4.20.

Please feel free to create a new section for 3.4.20, or let me take care of all items once 3.4.20 is released.

Copy link
Contributor Author

@mitake mitake Jul 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it, let me update this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ahrtr Updated, could you check?

@mitake mitake force-pushed the changelog-3.4-for-pr-14230 branch from fdee21c to f8e65cd Compare July 24, 2022 14:33
Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>
@mitake mitake force-pushed the changelog-3.4-for-pr-14230 branch from f8e65cd to 97222a5 Compare July 24, 2022 14:34
@codecov-commenter
Copy link

Codecov Report

Merging #14231 (97222a5) into main (6d27a42) will decrease coverage by 0.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #14231      +/-   ##
==========================================
- Coverage   75.58%   75.08%   -0.51%     
==========================================
  Files         456      456              
  Lines       36919    36919              
==========================================
- Hits        27904    27719     -185     
- Misses       7299     7450     +151     
- Partials     1716     1750      +34     
Flag Coverage Δ
all 75.08% <ø> (-0.51%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
server/auth/simple_token.go 80.00% <0.00%> (-8.47%) ⬇️
client/pkg/v3/fileutil/lock_linux.go 72.22% <0.00%> (-8.34%) ⬇️
client/pkg/v3/tlsutil/tlsutil.go 83.33% <0.00%> (-8.34%) ⬇️
server/storage/mvcc/watchable_store.go 85.14% <0.00%> (-8.34%) ⬇️
client/v3/concurrency/key.go 80.00% <0.00%> (-8.00%) ⬇️
server/etcdserver/api/v3rpc/interceptor.go 75.52% <0.00%> (-7.30%) ⬇️
client/v3/leasing/util.go 91.66% <0.00%> (-6.67%) ⬇️
raft/rafttest/node.go 95.00% <0.00%> (-5.00%) ⬇️
server/storage/wal/file_pipeline.go 90.69% <0.00%> (-4.66%) ⬇️
server/proxy/grpcproxy/watch.go 92.48% <0.00%> (-4.05%) ⬇️
... and 25 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thank you @mitake

@ahrtr ahrtr merged commit 4977877 into etcd-io:main Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants